-
Notifications
You must be signed in to change notification settings - Fork 831
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Some tidying and build fixes #1984
Conversation
Hey @mhamilton723 👋! We use semantic commit messages to streamline the release process. Examples of commit messages with semantic prefixes:
To test your commit locally, please follow our guild on building from source. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary by GPT-4
The changes in this commit include:
- Updating the FormRecognizerV3Suite test to include the "InvoiceTotal" field.
- Adding the "black" and "black[jupyter]" packages to the environment.yml file.
- Removing the .pre-commit-config.yaml file and hooks/pre-commit file.
- Updating some notebook files to fix formatting issues.
- Updating the pipeline.yaml file to use a conda environment instead of pip for installing dependencies.
- Removing the requirements.txt file since it is no longer needed.
- Updating the _Train.md documentation file to use a smaller dataset size for examples.
Overall, these changes seem to be focused on improving code quality and consistency, as well as updating some dependencies and documentation examples.
Suggestions
No suggestions are needed as the changes in this PR seem appropriate.
998fb78
to
1eac69d
Compare
1eac69d
to
be72fae
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov Report
@@ Coverage Diff @@
## master #1984 +/- ##
==========================================
+ Coverage 86.30% 86.98% +0.68%
==========================================
Files 305 305
Lines 15993 15993
Branches 839 839
==========================================
+ Hits 13802 13912 +110
+ Misses 2191 2081 -110 |
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
37c764d
to
0280f7f
Compare
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
Acrolinx ScorecardsA minimum Acrolinx score of 80 is required. Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.
More information about Acrolinx |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Related Issues/PRs
#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?
Does this PR change any dependencies?
Does this PR add a new feature? If so, have you added samples on website?
website/docs/documentation
folder.Make sure you choose the correct class
estimators/transformers
and namespace.DocTable
points to correct API link.yarn run start
to make sure the website renders correctly.<!--pytest-codeblocks:cont-->
before each python code blocks to enable auto-tests for python samples.WebsiteSamplesTests
job pass in the pipeline.